Skip to content

Conversation

@smitterl
Copy link
Contributor

@smitterl smitterl commented Jan 22, 2026

Reverts #6717
Merged but depends on avocado-framework/avocado-vt#4280 which hasn't been merged yet, so revert.

Summary by CodeRabbit

  • Tests
    • Simplified virtual network migration test configuration.
    • Enhanced test infrastructure for improved bridge interface handling during network migration scenarios.
    • Improved consistency and readability of test code through standardized formatting.

✏️ Tip: You can customize this high-level summary in your review settings.

@smitterl smitterl merged commit 87fab65 into master Jan 22, 2026
4 of 5 checks passed
@coderabbitai
Copy link

coderabbitai bot commented Jan 22, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

This change updates a virtual network migration test configuration and implementation. The configuration file removes a remote_host_iface line. The Python source file refactors formatting consistency (quote normalization), adds computation and propagation of remote_host_iface through setup and teardown paths for linux bridge operations, and updates corresponding function calls to pass this parameter. Changes maintain existing functionality while improving code consistency and parameter tracking.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

✨ Finishing touches
  • 📝 Generate docstrings

Comment @coderabbitai help to get the list of available commands and usage tips.

@smitterl smitterl deleted the revert-6717-vn_add_remote_iface_config branch January 22, 2026 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants